fix(sync): reconcile cached state inspection onto current main - #860
fix(sync): reconcile cached state inspection onto current main#860KooshaPari wants to merge 12 commits into
Conversation
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains; the previously reported recovery-guidance and cached-state telemetry issues are fixed on the current head. Reviews (2): Last reviewed commit: "test(e2e): give expanded matrix realisti..." | Re-trigger Greptile |
|
Speaking as Kun's firstmate: First look at HEAD Diff review agrees with Greptile P1s before merge:
Otherwise the read-only cached inspect + deferred merge-tree proof still looks like the right fix for #856. Waiting on those presenter/telemetry fixes (and green hosted CI) before merge. Please close #849 when this is the active path. VISION.md:
|
|
Current head |
|
Integration scope has been corrected: this work is now contained entirely in the KooshaPari fork. The preserved successor review is KooshaPari#5 at the same exact head |
Supersedes the implementation path of #849 without modifying that preserved PR or its source branch.
This branch is a clean, provenance-preserving
-xreplay of #849's seven commits onto current upstreammain(06c6504). It retains the cached-state CLI, the non-mutating bare-gate inspection contract, explicit custody-recovery guidance, and the regression coverage that caught the prior hidden Git-object write.Validation on this exact successor:
make lintinternal/branchsync,internal/pipeline/..., andinternal/cli#849 remains preserved because its sole hosted e2e result hit a transient Git maintenance-lock template-copy race and cannot be rerun by this account. This successor requires a fresh hosted review/CI matrix and human approval before any merge claim.